Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

sqs/sqs.go (1)

25-34: Move parameter clamping outside closures to eliminate mutation of captured values.

The three Option builders (WithVisibilityTimeout, WithMaxMessages, WithWaitTimeSeconds) mutate captured parameters inside their closures. While the current usage pattern is sequential, this design creates a potential concurrency hazard if an Option instance is ever reused or shared across goroutines. Clamp the parameters before returning the closure instead.

Also applies to: lines 40–48, 54–62

♻️ Proposed refactor

 func WithVisibilityTimeout(seconds int64) Option {
+	if seconds < 0 {
+		seconds = 0
+	}
+	if seconds > 43200 {
+		seconds = 43200
+	}
 	return func(c *Client) {
-		if seconds < 0 {
-			seconds = 0
-		}
-		if seconds > 43200 {
-			seconds = 43200
-		}
 		c.visibilityTimeout = seconds
 	}
 }

 func WithMaxMessages(n int64) Option {
+	if n < 1 {
+		n = 1
+	}
+	if n > 10 {
+		n = 10
+	}
 	return func(c *Client) {
-		if n < 1 {
-			n = 1
-		}
-		if n > 10 {
-			n = 10
-		}
 		c.maxMessages = n
 	}
 }

 func WithWaitTimeSeconds(seconds int64) Option {
+	if seconds < 0 {
+		seconds = 0
+	}
+	if seconds > 20 {
+		seconds = 20
+	}
 	return func(c *Client) {
-		if seconds < 0 {
-			seconds = 0
-		}
-		if seconds > 20 {
-			seconds = 20
-		}
 		c.waitTimeSeconds = seconds
 	}
 }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@sqs/sqs.go` around lines 25 - 34, The Option builders (WithVisibilityTimeout,
WithMaxMessages, WithWaitTimeSeconds) currently clamp their input parameters
inside the returned closure which mutates captured variables and can cause
concurrency hazards if the Option is reused; move the clamping logic out of the
closure by normalizing the input (apply min/max limits) before returning the
func(c *Client) closure so the closure only assigns the already-clamped value to
fields (visibilityTimeout, maxMessages, waitTimeSeconds) without mutating
captured variables.